Skip to content

FEAT: Adding descriptions to Techniques#2167

Merged
rlundeen2 merged 3 commits into
microsoft:mainfrom
rlundeen2:rlundeen2-attack-technique-descriptions
Jul 15, 2026
Merged

FEAT: Adding descriptions to Techniques#2167
rlundeen2 merged 3 commits into
microsoft:mainfrom
rlundeen2:rlundeen2-attack-technique-descriptions

Conversation

@rlundeen2

@rlundeen2 rlundeen2 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Description

Adds an optional description field to attack technique factories, so each technique in pyrit/setup carries a concise summary of what it does.

  • New keyword-only description: str | None = None on AttackTechniqueFactory.__init__ (and forwarded through with_simulated_conversation), exposed via a description property.
  • Deliberately excluded from _build_identifier — it's decorative metadata and must not change a technique's behavioral identity hash.
  • Populated a concise description for every technique in the core, extra, and airt catalogs.

Right now these are informational only (as the title says, currently useless / decorative); a follow-up could surface them in registry/CLI listings. But even just as informational, I was missing some of the description that we're losing as we're migrating attack classes to techniques. I think there needs to be a place where you can see what the technique is doing with english.

Add an optional description field to AttackTechniqueFactory (decorative metadata, excluded from the behavioral identity hash) and populate a concise description for every technique in the core, extra, and airt catalogs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@rlundeen2 rlundeen2 changed the title FEAT: Adding useless descriptions to Techniques FEAT: Adding descriptions to Techniques Jul 13, 2026
@behnam-o behnam-o self-requested a review July 14, 2026 21:42
Comment thread pyrit/setup/initializers/techniques/core.py Outdated
…nique-descriptions

# Conflicts:
#	pyrit/setup/initializers/techniques/core.py

@jsong468 jsong468 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want these shown in our attack_techniques notebook or is that too verbose?

Comment thread pyrit/setup/initializers/techniques/core.py Outdated
Address review nits on PR microsoft#2167: open every description with an active verb and use consistent 'Frames' wording for the role_play variants (was cast/staged/framed synonyms), and rewrite the crescendo entries from noun phrases to active verbs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@rlundeen2 rlundeen2 enabled auto-merge July 14, 2026 23:43
@rlundeen2 rlundeen2 added this pull request to the merge queue Jul 14, 2026
Merged via the queue into microsoft:main with commit 1434fe4 Jul 15, 2026
53 checks passed
@rlundeen2 rlundeen2 deleted the rlundeen2-attack-technique-descriptions branch July 15, 2026 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants